Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding Polymarket Cookbook #858

Merged
merged 7 commits into from
Jan 27, 2025
Merged

Adding Polymarket Cookbook #858

merged 7 commits into from
Jan 27, 2025

Conversation

MichaelMacaulay
Copy link
Contributor

No description provided.

@MichaelMacaulay MichaelMacaulay requested a review from a team as a code owner January 23, 2025 21:26
@benface
Copy link
Contributor

benface commented Jan 23, 2025

Hey @MichaelMacaulay, can you run pnpm check:fix please? 🙏

Also, I recommend these VS Code settings (requires the Prettier extension):

{
    "editor.formatOnSave": true,
    "editor.defaultFormatter": "esbenp.prettier-vscode"
}

Copy link
Contributor

@idalithb idalithb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Terrific add-in @MichaelMacaulay 😄
  • Minor comments, but I love the simplicity.
  • I think it might be useful to add a line or heading that let's users know whether this is a guide or tutorial.

website/pages/en/subgraphs/cookbook/polymarket.mdx Outdated Show resolved Hide resolved
website/pages/en/subgraphs/cookbook/polymarket.mdx Outdated Show resolved Hide resolved
website/pages/en/subgraphs/cookbook/polymarket.mdx Outdated Show resolved Hide resolved
website/pages/en/subgraphs/cookbook/polymarket.mdx Outdated Show resolved Hide resolved
@MichaelMacaulay MichaelMacaulay merged commit 6ac3474 into main Jan 27, 2025
4 checks passed
@MichaelMacaulay MichaelMacaulay deleted the Polymarket-cookbook branch January 27, 2025 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants